Skip to content

chore: format snippet metadata#14261

Merged
JoeWang1127 merged 1 commit intogoogleapis:mainfrom
JoeWang1127:chore/format-snippet
Mar 25, 2026
Merged

chore: format snippet metadata#14261
JoeWang1127 merged 1 commit intogoogleapis:mainfrom
JoeWang1127:chore/format-snippet

Conversation

@JoeWang1127
Copy link
Copy Markdown
Contributor

@JoeWang1127 JoeWang1127 commented Mar 25, 2026 •
edited
Loading

Format snippet metadata, prepare for librarian Go migration.

The change is made through the following commands:

# sort keys
find internal/ -name "snippet_metadata*.json" -exec sh -c 'jq -S "." "$1" > "$1.tmp" && mv "$1.tmp" "$1"' _ {} \;
# remove the newline at the end of file
find internal/ -name "snippet_metadata*.json" -exec perl -i -pe 'chomp if eof' {} +

gemini-code-assist[bot]
gemini-code-assist Bot reviewed Mar 25, 2026
View reviewed changes
Copy link
Copy Markdown
Contributor

@gemini-code-assist gemini-code-assist Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

This pull request reorders fields within JSON metadata files for various client libraries and their snippets. The changes appear to be purely cosmetic, likely due to an automated formatting or code generation process, and do not alter the data itself. No specific issues or improvement opportunities were identified in the review comments.

@JoeWang1127 JoeWang1127 enabled auto-merge (squash) March 25, 2026 19:18
@JoeWang1127 JoeWang1127 merged commit 9b18aaf into googleapis:main Mar 25, 2026
10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@shollyman shollyman shollyman approved these changes

+1 more reviewer

@gemini-code-assist gemini-code-assist[bot] gemini-code-assist[bot] left review comments

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@JoeWang1127 @shollyman
⚡